Skip to content

fix(ui): Add claim name for Custom SAML provider in <ConfigureSSO />#8586

Merged
LauraBeatris merged 3 commits into
mainfrom
laura/fix-attribute-mapping
May 18, 2026
Merged

fix(ui): Add claim name for Custom SAML provider in <ConfigureSSO />#8586
LauraBeatris merged 3 commits into
mainfrom
laura/fix-attribute-mapping

Conversation

@LauraBeatris
Copy link
Copy Markdown
Member

@LauraBeatris LauraBeatris commented May 18, 2026

Description

This PR fixes the attribute statements step for Custom SAML provider, where previously was displaying the claim value with an expression that is specific to Okta

It also fixes an issue on the test step, where it was always disabling the button, but the validation actually happens on click.

Custom SAML

Screenshot 2026-05-18 at 15 56 35

Okta

Screenshot 2026-05-18 at 15 39 49

Checklist

  • pnpm test runs as expected.
  • pnpm build runs as expected.
  • (If applicable) JSDoc comments have been added or updated for any package exports
  • (If applicable) Documentation has been updated

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

@LauraBeatris LauraBeatris self-assigned this May 18, 2026
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 18, 2026

🦋 Changeset detected

Latest commit: 0a8f3f3

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 20 packages
Name Type
@clerk/localizations Patch
@clerk/clerk-js Patch
@clerk/shared Patch
@clerk/ui Patch
@clerk/react Patch
@clerk/chrome-extension Patch
@clerk/expo Patch
@clerk/astro Patch
@clerk/backend Patch
@clerk/expo-passkeys Patch
@clerk/express Patch
@clerk/fastify Patch
@clerk/hono Patch
@clerk/msw Patch
@clerk/nextjs Patch
@clerk/nuxt Patch
@clerk/react-router Patch
@clerk/tanstack-react-start Patch
@clerk/testing Patch
@clerk/vue Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link
Copy Markdown

vercel Bot commented May 18, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
clerk-js-sandbox Ready Ready Preview, Comment May 18, 2026 6:57pm

Request Review

@LauraBeatris LauraBeatris requested a review from a team May 18, 2026 18:47
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 18, 2026

Open in StackBlitz

@clerk/astro

npm i https://pkg.pr.new/@clerk/astro@8586

@clerk/backend

npm i https://pkg.pr.new/@clerk/backend@8586

@clerk/chrome-extension

npm i https://pkg.pr.new/@clerk/chrome-extension@8586

@clerk/clerk-js

npm i https://pkg.pr.new/@clerk/clerk-js@8586

@clerk/dev-cli

npm i https://pkg.pr.new/@clerk/dev-cli@8586

@clerk/expo

npm i https://pkg.pr.new/@clerk/expo@8586

@clerk/expo-passkeys

npm i https://pkg.pr.new/@clerk/expo-passkeys@8586

@clerk/express

npm i https://pkg.pr.new/@clerk/express@8586

@clerk/fastify

npm i https://pkg.pr.new/@clerk/fastify@8586

@clerk/hono

npm i https://pkg.pr.new/@clerk/hono@8586

@clerk/localizations

npm i https://pkg.pr.new/@clerk/localizations@8586

@clerk/nextjs

npm i https://pkg.pr.new/@clerk/nextjs@8586

@clerk/nuxt

npm i https://pkg.pr.new/@clerk/nuxt@8586

@clerk/react

npm i https://pkg.pr.new/@clerk/react@8586

@clerk/react-router

npm i https://pkg.pr.new/@clerk/react-router@8586

@clerk/shared

npm i https://pkg.pr.new/@clerk/shared@8586

@clerk/tanstack-react-start

npm i https://pkg.pr.new/@clerk/tanstack-react-start@8586

@clerk/testing

npm i https://pkg.pr.new/@clerk/testing@8586

@clerk/ui

npm i https://pkg.pr.new/@clerk/ui@8586

@clerk/upgrade

npm i https://pkg.pr.new/@clerk/upgrade@8586

@clerk/vue

npm i https://pkg.pr.new/@clerk/vue@8586

commit: 0a8f3f3

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 18, 2026

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro

Run ID: df0a895d-ac3d-4f89-b5b2-48b8e0c940e4

📥 Commits

Reviewing files that changed from the base of the PR and between d4fc01a and 0a8f3f3.

📒 Files selected for processing (2)
  • .changeset/fancy-zoos-mate.md
  • packages/ui/src/components/ConfigureSSO/steps/TestConfigurationStep.tsx
💤 Files with no reviewable changes (1)
  • packages/ui/src/components/ConfigureSSO/steps/TestConfigurationStep.tsx
🚧 Files skipped from review as they are similar to previous changes (1)
  • .changeset/fancy-zoos-mate.md

📝 Walkthrough

Walkthrough

This PR extends the component to add an Okta-specific claim value column to the attribute mapping table. The localization type definition gains a new claimValue field, and en-US strings are updated with the column header and example claim values. The ConfigureStep component conditionally renders the claim value column only when the provider is Okta, using the new localization keys. The TestConfigurationStep component has its Continue button's isDisabled prop removed, letting button state be controlled purely by the onClick handler and isLoading flag. Patch releases are recorded for four affected packages.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

  • clerk/javascript#8535: The main PR’s updates to packages/ui/src/components/ConfigureSSO/steps/ConfigureStep.tsx (adding the Okta-specific “claim value” column and wiring oktaClaimValue/isOkta) build directly on the retrieved PR’s Okta Configure step UI/table work in the same ConfigureStep flow.
  • clerk/javascript#8564: Both PRs modify the same <ConfigureSSO /> configure-step stack—updating the shared SSO localization/type schema and the packages/ui/src/components/ConfigureSSO/steps/ConfigureStep.tsx UI with provider-specific (Okta/custom SAML) rendering—so the main PR’s new Okta “claim value” column fits into the custom/Okta SAML support introduced by the retrieved PR.
  • clerk/javascript#8544: The main PR’s change to packages/ui/src/components/ConfigureSSO/steps/TestConfigurationStep.tsx (removing the isDisabled/isConnectionActive continue-button wiring) directly overlaps the retrieved PR’s newly added/rewired ConfigureSSO test step footer/continue behavior.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main fix: adding the correct claim name for Custom SAML provider in the ConfigureSSO component, which aligns with the core changes across all modified files.
Description check ✅ Passed The description is clearly related to the changeset, explaining the Custom SAML fix and the test step issue, with visual evidence via screenshots showing the before/after UI for both Custom SAML and Okta providers.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

@LauraBeatris LauraBeatris merged commit 95f6c2f into main May 18, 2026
45 checks passed
@LauraBeatris LauraBeatris deleted the laura/fix-attribute-mapping branch May 18, 2026 19:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants